Skip to content

chore: remove font antialiasing#3806

Merged
emir-karabeg merged 2 commits intosimstudioai:stagingfrom
adithyaakrishna:chore/aa
Mar 27, 2026
Merged

chore: remove font antialiasing#3806
emir-karabeg merged 2 commits intosimstudioai:stagingfrom
adithyaakrishna:chore/aa

Conversation

@adithyaakrishna
Copy link
Copy Markdown
Contributor

Summary

  • Fixes issue caused antialiasing

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation
  • Other: ___________

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@cursor
Copy link
Copy Markdown

cursor bot commented Mar 27, 2026

PR Summary

Low Risk
Low risk: removes a single global Tailwind antialiased style on body, which may slightly change font rendering but doesn't affect logic or data flow.

Overview
Removes the global body { @apply antialiased; } rule from globals.css, eliminating forced font antialiasing across the app.

Written by Cursor Bugbot for commit ff00e32. This will update automatically on new commits. Configure here.

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Mar 27, 2026 6:25pm

Request Review

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Mar 27, 2026

Greptile Summary

This PR removes the @apply antialiased rule from the body selector in globals.css, reverting global font rendering from WebKit's antialiased smoothing back to the browser's default (subpixel antialiasing). The change is a 3-line deletion with no logic, API, or dependency changes.\n\n- The removal is intentional and affects the visual appearance of all text across the application.\n- Because this is a body-level global style, globals.css is the only appropriate location — it cannot be scoped to a local component file (the custom rule to avoid editing globals.css applies when a change could be localised, which is not the case here).\n- The PR description ("Fixes issue caused antialiasing") is terse; it would be helpful to briefly describe the visual artifact that prompted the change, so future contributors understand why the default was overridden and then removed.

Confidence Score: 5/5

Safe to merge — a single global CSS deletion with no logic or API changes and no regressions.

The change is a pure style removal with well-understood visual impact. No P0 or P1 issues exist; the only note is a P2 suggestion to improve the PR description for historical context.

No files require special attention.

Important Files Changed

Filename Overview
apps/sim/app/_styles/globals.css Removes the antialiased Tailwind utility from the body selector — a 3-line deletion that reverts global font smoothing to the browser default. No regressions introduced.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["Browser renders text"] --> B{"font-smoothing applied?"}
    B -- "Before PR\n(antialiased on body)" --> C["-webkit-font-smoothing: antialiased\n-moz-osx-font-smoothing: grayscale\n→ thinner, lighter text"]
    B -- "After PR\n(no override)" --> D["Browser default\n(-webkit-font-smoothing: auto)\n→ subpixel antialiasing"]
Loading

Reviews (1): Last reviewed commit: "chore: remove antialiasing" | Re-trigger Greptile

@emir-karabeg emir-karabeg merged commit e698f9f into simstudioai:staging Mar 27, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants